@copilot review this MR

@copilot review this MR

Here's the review of the PR:

Overall: ✅ Changes are correct and well-structured. The core goal — guarding every Crashlytics call against a GMS cold-start crash — is fully achieved.

FirebaseUtils.kt

• All 5 wrappers (safeRecordException, safeLog, safeSetCustomKey ×3) follow a consistent try/catch pattern with a Log.w fallback. ✅
• FirebaseCrashlytics import is now exclusively here — no other file needs it. ✅
• Calling getInstance() on every invocation (rather than caching it) is intentional and correct: GMS availability can change across the lifetime of the service, so a fresh lookup ensures the guard is always meaningful. ✅

Call-site changes (5 files)

• DashTuneMusicService, DashTuneSessionCallback, SignInViewModel, AlbumArtContentProvider, MediaRepository: all direct .log(), .setCustomKey(), and .recordException() calls replaced; FirebaseCrashlytics import removed in each. ✅
• The separate compilation-fix commit (aaa2f8a) correctly removes the redundant FirebaseUtils import from AlbumArtContentProvider (same package — no import needed) and restores the missing OkHttpClient import. ✅

Minor observations (no action needed)

• safeLog("Sync started") is called both in DashTuneSessionCallback.sync() and MediaRepository.sync(), so a sync will log twice. This is harmless and consistent with the existing verbosity style.
• The pingServer error-handling in SignInViewModel extracts the URL host via a nested try/catch before recording the exception — a little verbose, but safe and the intent is clear.

No regressions, no unguarded Crashlytics paths remaining. LGTM.